Skip to content

feat: Simulation upgrades - #120

Merged
TEJASNARAYANS merged 10 commits into
mainfrom
simulation-upgrades
Sep 4, 2026
Merged

feat: Simulation upgrades#120
TEJASNARAYANS merged 10 commits into
mainfrom
simulation-upgrades

Conversation

@TEJASNARAYANS

@TEJASNARAYANS TEJASNARAYANS commented Aug 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

What Changed?

Briefly describe what this PR changes.

Why?

Explain the problem this solves and why this approach was chosen.

How to Test?

List clear steps for reviewers to verify the change.

Release Label

Select one semantic version bump intent for this PR:

  • major - breaking change, next release bumps major version
  • minor - backward-compatible feature, next release bumps minor version
  • fix - backward-compatible bug fix, next release bumps patch version
  • No label (defaults to patch release)

If you do not have permission to apply labels, mention the intended release label here and a maintainer will set it.

Checklist

  • I have read the CONTRIBUTING.md guide.
  • My code follows the project's style guidelines.
  • I have added tests that prove my fix is effective or my feature works.
  • I have updated documentation where needed.

@TEJASNARAYANS TEJASNARAYANS added the fix Fix version release label Aug 27, 2026
@TEJASNARAYANS

TEJASNARAYANS commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

@greptile-apps

@greptile-apps

greptile-apps Bot commented Aug 27, 2026 •
edited
Loading

Copy link
Copy Markdown

Greptile Summary

This PR substantially expands simulation workflows, including structured test-agent templates, ambient persona audio, evaluator result clustering, recording lifecycle changes, and corresponding frontend experiences.

  • Adds structured test-agent prompt and first-message generation and persistence.
  • Adds ambient-noise library, upload, preview, and mixing support for personas.
  • Adds evaluator-result metric clustering, aggregation, background jobs, and reporting UI.
  • Extends voice, telephony, recording, and transcript processing.

Confidence Score: 3/5

The PR does not appear safe to merge until ambient-audio uploads enforce their size limit without first materializing an unbounded request file in worker memory.

Both current upload paths still execute await file.read() before checking the 10 MiB maximum, so an authenticated oversized multipart upload can consume memory proportional to the request and terminate an API worker.

Files Needing Attention: app/api/v1/routes/personas.py

Important Files Changed
Filename Overview
app/api/v1/routes/personas.py Adds ambient-audio library and persona upload endpoints; the previously reported pre-validation memory exposure remains outstanding.
app/services/audio/ambient_mixer.py Implements ambient-audio decoding, looping, gain adjustment, and speech mixing.
app/services/testing/test_agent_template.py Introduces normalized structured templates and prompt assembly for generated test agents.
app/api/v1/routes/evaluator_result_metric_clusters.py Adds evaluator-result metric-cluster APIs and registers them ahead of the dynamic result route.
app/workers/tasks/generate_evaluator_result_metric_clusters.py Adds asynchronous generation of metric clusters for evaluator-result scopes.
frontend/src/pages/personas/AmbientNoiseLibraryPanel.tsx Adds the persona ambient-noise library management interface.
frontend/src/components/metricClusters/MetricClustersPanel.tsx Adds the frontend entry point for metric-cluster reports and generation controls.

Reviews (2): Last reviewed commit: "feat: updating dashboards and results" | Re-trigger Greptile

greptile-apps[bot]
greptile-apps Bot reviewed Aug 27, 2026
View reviewed changes
Comment thread app/api/v1/routes/personas.py Outdated
root and others added 6 commits August 29, 2026 21:07
@TEJASNARAYANS

TEJASNARAYANS commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@greptile-apps

@greptile-apps

greptile-apps Bot commented Sep 4, 2026

Copy link
Copy Markdown

Too many files changed for review (189 files, 100 file limit).

Bypass the limit by tagging @greptile-apps to review.

@TEJASNARAYANS
TEJASNARAYANS merged commit 6b9e599 into main Sep 4, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments

Assignees

No one assigned

Labels

fix Fix version release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@TEJASNARAYANS